Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adding some basic printing #37

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Adding some basic printing #37

wants to merge 18 commits into from

Conversation

zanellia
Copy link
Contributor

@zanellia zanellia commented Jul 31, 2018 •
edited
Loading

Adding basic printing. Setting arg->printlevel = 1 the progress towards the solution can be monitored.

zanellia added 18 commits July 19, 2018 11:18
@zanellia
adding getting_started.c
2b3a376
@zanellia
fix Makefile.rule
44860c8
@zanellia
adding print level field to args, retabbing
70c0ced
@zanellia
adding print level field to args for ocp_qp too, retabbing
c652c27
@zanellia
added basic printing
9325e9a
@zanellia
resolved conflict on getting_started.c with --ours
3436b2a
@zanellia
adding printing for absolute formulation
879900e
@zanellia
added printing for dense abs
b3113e6
@zanellia
retabbing x_ocp_qp_ipm.c
a8a10af
@zanellia
retabbing x_dense_qp_ipm.c
37671fe
@zanellia
retabbing hpipm_d_dense_qp_ipm.h
de0a1b6
@zanellia
Revert "retabbing hpipm_d_dense_qp_ipm.h"
c29c1c7 
This reverts commit de0a1b6.
@zanellia
Revert "retabbing x_dense_qp_ipm.c"
85b2d75 
This reverts commit 37671fe.
@zanellia
Revert "adding print level field to args for ocp_qp too, retabbing"
6af3b74 
This reverts commit c652c27.
@zanellia
Revert "adding print level field to args, retabbing"
c5fe952 
This reverts commit 70c0ced.
@zanellia
readded print level after reverting
a424987
@zanellia
merging with blessed/master
20e1f63
@zanellia
merged with blessed/master
1117f5c
@giaf
Copy link
Owner

giaf commented Aug 1, 2018

I would add a makefile flag to use C pre-processing to remove all printing dependencies, like in BLASFEO
https://github.com/giaf/blasfeo/blob/master/Makefile.rule#L86
such that it is possible to compile the library without any printf in the code.

And I would do the same for all 3 QP solvers: dense, ocp and tree_ocp

@zanellia zanellia changed the title adding some basic printing Adding some basic printing Aug 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zanellia @giaf